Skip to content

Fix cozy-ui-plus path and package.json issues#2983

Merged
zatteo merged 2 commits intomasterfrom
fix/peer-dep-cozy-ui-plus
Apr 7, 2026
Merged

Fix cozy-ui-plus path and package.json issues#2983
zatteo merged 2 commits intomasterfrom
fix/peer-dep-cozy-ui-plus

Conversation

@zatteo
Copy link
Copy Markdown
Member

@zatteo zatteo commented Apr 7, 2026

Summary by CodeRabbit

  • Chores
    • Updated internal dependency management and import paths to improve code organization and maintainability.

zatteo added 2 commits April 7, 2026 13:36
An old version of cozy-interapp was hardcoded in cozy-ui-plus making
IntentDialogOpener unable to return a result. Let's set it as peerDep
as our other internal deps.

BREAKING CHANGE: You need cozy-interapp >= 0.17.1
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 7, 2026

Walkthrough

The cozy-ui-plus package was updated to modify dependencies and reorganize component imports. The cozy-interapp package version was bumped from ^0.5.4 to ^0.17.1, converted from a direct dependency to a peer dependency in package.json. In the FilePicker module, the LoadMore component import source was changed from a local module to the cozy-ui transpiled version, and the getCompliantTypes import path was adjusted to reference a local helpers module instead of the parent directory.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main changes: fixing import paths in FilePicker modules and updating package.json dependencies for cozy-interapp.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/peer-dep-cozy-ui-plus

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick comments (1)
packages/cozy-ui-plus/package.json (1)

70-70: Constrain the cozy-interapp peer range to the current minor line.

The range >=0.17.1 is too broad for a 0.x package. In pre-1.0 semver, even minor bumps (e.g., 0.18.0) can introduce breaking changes. Use ^0.17.1 (expands to >=0.17.1 <0.18.0) or an explicit upper bound >=0.17.1 <0.18.0 to avoid accidental breaking upgrades.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@packages/cozy-ui-plus/package.json` at line 70, The peer dependency range for
"cozy-interapp" is too permissive (">=0.17.1"); update the version range in
package.json to constrain to the current minor line for the 0.x series—replace
">=0.17.1" with either "^0.17.1" or the explicit upper bound ">=0.17.1 <0.18.0"
to prevent accidental incompatible upgrades.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@packages/cozy-ui-plus/package.json`:
- Line 70: The peer dependency range for "cozy-interapp" is too permissive
(">=0.17.1"); update the version range in package.json to constrain to the
current minor line for the 0.x series—replace ">=0.17.1" with either "^0.17.1"
or the explicit upper bound ">=0.17.1 <0.18.0" to prevent accidental
incompatible upgrades.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ef08b089-1d71-4922-b52a-a0290ff13cb7

📥 Commits

Reviewing files that changed from the base of the PR and between 8224060 and a6536d5.

⛔ Files ignored due to path filters (1)
  • yarn.lock is excluded by !**/yarn.lock, !**/*.lock
📒 Files selected for processing (3)
  • packages/cozy-ui-plus/package.json
  • packages/cozy-ui-plus/src/FilePicker/FilePickerBody.jsx
  • packages/cozy-ui-plus/src/FilePicker/index.jsx

@zatteo zatteo merged commit 27bd85c into master Apr 7, 2026
3 checks passed
@zatteo zatteo deleted the fix/peer-dep-cozy-ui-plus branch April 7, 2026 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants